Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIRROR] Fixes Alien Hunter's Pounce Visuals #1855

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#829
Original PR: tgstation/tgstation#81322

About The Pull Request

Currently on live, when an alien hunter goes for a pounce, once the pounce is completed the hunter will rise quite a distance slowly from the floor as setting body_position_pixel_y_offset now is done with animation as opposed to being instant. This looked extremely stupid, so I went and fixed it. Initially, I was going to add some form of trait to cancel out the smoothing effect from a shifting y offset, until I looked at the sprite for alien pouncing and realized it was 96x96 for no reason. For reference, 96x96 is the size of Bubblegum's sprites.

So, I just made the sprites 48x32, got rid of the unused sprites (sentinel and drone both had pounce sprites but neither ever had that ability iirc) and adjusted some of the pounce ability's internal offset stuff to compensate and it looks perfectly fine now. Somehow made the image file larger by 243 bytes according to Github but I guess it is what it is.

Why It's Good For The Game

The alien hunter ascending from 2 tiles below where it actually was whenever it pounced was incredibly jarring. This fixes the problem.

Changelog

🆑 IndieanaJones
fix: Alien Hunter's pounce ability no longer has any weird offset issues once the pounce is completed.
/:cl:

* Fixes Alien Hunter's Pounce Visuals (#81322)

## About The Pull Request

Currently on live, when an alien hunter goes for a pounce, once the
pounce is completed the hunter will rise quite a distance slowly from
the floor as setting body_position_pixel_y_offset now is done with
animation as opposed to being instant. This looked extremely stupid, so
I went and fixed it. Initially, I was going to add some form of trait to
cancel out the smoothing effect from a shifting y offset, until I looked
at the sprite for alien pouncing and realized it was 96x96 for no
reason. For reference, 96x96 is the size of Bubblegum's sprites.

So, I just made the sprites 48x32, got rid of the unused sprites
(sentinel and drone both had pounce sprites but neither ever had that
ability iirc) and adjusted some of the pounce ability's internal offset
stuff to compensate and it looks perfectly fine now. Somehow made the
image file larger by 243 bytes according to Github but I guess it is
what it is.

## Why It's Good For The Game

The alien hunter ascending from 2 tiles below where it actually was
whenever it pounced was incredibly jarring. This fixes the problem.

## Changelog
:cl:
fix: Alien Hunter's pounce ability no longer has any weird offset issues
once the pounce is completed.
/:cl:

* Fixes Alien Hunter's Pounce Visuals

---------

Co-authored-by: IndieanaJones <[email protected]>
@Iajret Iajret merged commit 0204a8d into master Feb 8, 2024
24 checks passed
AnywayFarus added a commit that referenced this pull request Feb 8, 2024
@Iajret Iajret deleted the upstream-mirror-829 branch February 8, 2024 08:47
Iajret pushed a commit that referenced this pull request Apr 7, 2024
* Maint Mania dust runtime fix (#82487)

## About The Pull Request

This removes some dust from a space tile that was throwing stack_traces
every time the Maint Mania map was loaded up.
## Why It's Good For The Game

Less clutter while looking for an entirely unrelated runtime regarding
deathmatch.
## Changelog
:cl: Rhials
fix: Sweeps a tile of dust off of the Maint Mania deathmatch map, which
was causing errors.
/:cl:

* Maint Mania dust runtime fix

---------

Co-authored-by: Rhials <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants